Skip to content

Conversation

masseyke
Copy link
Member

(cherry picked from commit 9e631d8)

This backports #4416 to 8.19.

Copy link
Contributor

github-actions bot commented Sep 25, 2025

Following you can find the validation changes against the target branch for the APIs.

API Status Request Response
indices.get_data_stream_settings 🟠 → 🟢 Missing type → 8/8 Missing type → 8/8
indices.put_data_stream_settings 🟠 → 🔴 Missing type → 6/8 Missing type → 8/8

You can validate these APIs yourself by using the make validate target.

@pquentin pquentin changed the title Adding specification for the new data stream settings APIs (#4416) [8.19] Adding specification for the new data stream settings APIs (#4416) Sep 26, 2025
Copy link
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM.

I realize this is only a backport, but for the record the failure is caused by this YAML test where index.lifecycle.name and index.lifecycle.prefer_ilm are set to null:

  - do:
      indices.put_data_stream_settings:
        name: my-component-only-data-stream-1
        body:
          index:
            lifecycle:
              name: null
              prefer_ilm: null

https://github.com/elastic/elasticsearch/blob/734a6b70f445d865e64ec2e05e5f308fdaf1f43e/modules/data-streams/src/yamlRestTest/resources/rest-api-spec/test/data_stream/240_data_stream_settings.yml#L464-L470

(And the other issue is that applied_to_data_stream_and_write_indices is missing from the specification. But that does not affect 8.19.)

@pquentin pquentin merged commit 636ab1c into 8.19 Sep 27, 2025
7 checks passed
@pquentin pquentin deleted the backport-4416-to-8.19 branch September 27, 2025 05:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants